-
Notifications
You must be signed in to change notification settings - Fork 105
vllm - Add initial set of metrics #7285
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@rzabarazesh is attempting to deploy a commit to the Meta Open Source Team on Vercel. A member of the Team first needs to authorize it. |
74cf17a
to
e7ddb4b
Compare
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PyTorch and test-infra uses a tool called lintrunner to all linters, our version of pre-commit. You want to install it https://pypi.org/project/lintrunner and Also, the failure in https://github.com/pytorch/test-infra/actions/runs/18228918765/job/51907208089?pr=7285 can be solved easily by running |
e7ddb4b
to
aed2831
Compare
aed2831
to
1d728cf
Compare
} | ||
} | ||
|
||
const options: EChartsOption = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you want to choose only some legends ? right now the
{ name: "Success" },
{ name: "Failed" },
{ name: "Canceled" },
are not clickable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I understood this one. Do you mean the data points? Or that the legend itself isn't clickable?
Adds metrics for both CI runtime and code review cycle
Updated to now add reliability metrics as well.